Skip to content

Update continuous_link_flap.yml to fixed the test failed issue #1016

Closed
vincent201881 wants to merge 1 commit intosonic-net:masterfrom
vincent201881:patch-7
Closed

Update continuous_link_flap.yml to fixed the test failed issue #1016
vincent201881 wants to merge 1 commit intosonic-net:masterfrom
vincent201881:patch-7

Conversation

@vincent201881
Copy link

Description of PR

Summary:
Fixes # (issue)

  1. re-back the Commented out statement to provide parameter "interface" for the continuous_peer_link_flap_helper.yml script;
  2. added the pause statement for waiting the DUT all status up.

Type of change

  • [√] Bug fix
  • [] Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

How did you verify/test it?

    Verified the continuous_link_flap case on my own testbed environment.

The result:

TASK [test : debug] ************************************************************
task path: /var/clsnet/git-sw-csa/sonic-mgmt/mymgmt/sonic-mgmt/ansible/roles/test/tasks/test_sonic_by_testname.yml:44
Wednesday 17 July 2019 09:11:40 +0000 (0:00:00.651) 0:46:29.563 ********
ok: [cel-seastone-01] => {
"msg": [
"!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!",
"!!!!!!!!!!!!!!!!!!!! end running test continuous_link_flap !!!!!!!!!!!!!!!!!!!!!!",
"!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
]
}

TASK [test : include] **********************************************************
task path: /var/clsnet/git-sw-csa/sonic-mgmt/mymgmt/sonic-mgmt/ansible/roles/test/tasks/main.yml:8
Wednesday 17 July 2019 09:11:41 +0000 (0:00:00.367) 0:46:29.931 ********
skipping: [cel-seastone-01] => {"changed": false, "skip_reason": "Conditional check failed", "skipped": true}

PLAY ***************************************************************************
skipping: no hosts matched

PLAY RECAP *********************************************************************
cel-seastone-01 : ok=2604 changed=949 unreachable=0 failed=0

Wednesday 17 July 2019 09:11:41 +0000 (0:00:00.209) 0:46:30.140 ********

TASK: test : include continuous_peer_link_flap_helper.yml ------------- 221.24s
TASK: test : include continuous_peer_link_flap_helper.yml ------------- 170.51s
TASK: test : include continuous_peer_link_flap_helper.yml ------------- 113.16s
TASK: test : include continuous_link_flap_helper.yml ------------------- 76.90s
TASK: test : perform config reload ------------------------------------- 74.15s
TASK: test : wait 60 secs so that BGP routes are relearned ------------- 60.38s
TASK: test : include continuous_link_flap_helper.yml ------------------- 53.84s
TASK: test : wait 60 secs ---------------------------------------------- 30.34s
TASK: test : include continuous_link_flap_helper.yml ------------------- 15.87s
TASK: test : wait 15 secs ---------------------------------------------- 15.38s
TASK: test : set_fact --------------------------------------------------- 5.90s
TASK: test : watch orchagent CPU utilization ---------------------------- 3.15s
TASK: test : watch orchagent CPU utilization ---------------------------- 2.97s
TASK: test : watch orchagent CPU utilization ---------------------------- 2.96s
TASK: test : Watch memory status ---------------------------------------- 2.91s
TASK: test : Watch memory status ---------------------------------------- 2.90s
TASK: test : watch orchagent CPU utilization ---------------------------- 2.84s
TASK: test : Watch memory status ---------------------------------------- 2.83s
TASK: test : watch orchagent CPU utilization ---------------------------- 2.69s
TASK: test : watch orchagent CPU utilization ---------------------------- 2.68s
clsnet@29750a2b930a:~/git-sw-csa/sonic-mgmt/mymgmt/sonic-mgmt/ansible$

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@vincent201881 vincent201881 deleted the patch-7 branch July 15, 2020 02:33
fraserg-arista pushed a commit to fraserg-arista/sonic-mgmt that referenced this pull request Feb 24, 2026
…worm (sonic-net#1016)

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:
Fixes # (issue)

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [x] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] New Test case
 - [ ] Skipped for non-supported platforms
- [ ] Test case improvement

### Back port request
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411

### Approach
#### What is the motivation for this PR?

The base network containers were on Jessie, which is out of date. Update this to Bookworm.

#### How did you do it?

#### How did you verify/test it?

#### Any platform specific information?

#### Supported testbed topology if it's a new test case?

### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
544e4c2 (origin/202012) [ci] Paralize azure pipeline (sonic-net#1044)
45b310d Enable SAI_SWITCH_ATTR_UNINIT_DATA_PLANE_ON_REMOVAL attribute (sonic-net#1016)
9e56674 Advance submodule SAI for 202012 branch (sonic-net#1032)

Signed-off-by: bingwang <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
1. remove container feature table
2. do not generate feature entry if the feature is not included
   in the image
3. rename ENABLE_* to INCLUDE_* for better clarity
4. rename feature status to feature state
5. [submodule]: update sonic-utilities

* 9700e45 2020-08-03 | [show/config]: combine feature and container feature cli (sonic-net#1015) (HEAD, origin/master, origin/HEAD) [lguohan]
* c9d3550 2020-08-03 | [tests]: fix drops_group_test failure on second run (sonic-net#1023) [lguohan]
* dfaae69 2020-08-03 | [lldpshow]: Fix input device is not a TTY error (sonic-net#1016) [Arun Saravanan Balachandran]
* 216688e 2020-08-02 | [tests]: rename sonic-utilitie-tests to tests (sonic-net#1022) [lguohan]

Signed-off-by: Guohan Lu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants